size-limit report 📦

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

e2e tests

⚠️ Some screenshots were failed. See Playwright Report.

Playwright Report

👀 Docs deployed

• ✅ Styleguide
• ✅ Storybook

Commit 2110f86

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.34%. Comparing base (f198947) to head (2110f86).
Report is 5 commits behind head on master.

@@           Coverage Diff           @@
##           master    #7736   +/-   ##
=======================================
  Coverage   95.34%   95.34%           
=======================================
  Files         377      377           
  Lines       11054    11059    +5     
  Branches     3673     3677    +4     
=======================================
+ Hits        10539    10544    +5     
  Misses        515      515           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@EldarMuhamethanov А что думаешь по поводу рассуждений @Kritolan в #7733 (comment)

В этом PR мы игнорируем DateInput, а там и правда closeOnChange отключается когда enableTime включен.
Мы намеренно не даём это делать в DateInput?

Если подумать, то кнопка "Готово" затем и нужна, чтобы иметь возможность закрыть календарь. Если её намеренно прячут, то остаётся только закрытие по клику снаружи, как если бы closeOnChange было false у календаря без времени.

@EldarMuhamethanov А что думаешь по поводу рассуждений @Kritolan в #7733 (comment)

В этом PR мы игнорируем DateInput, а там и правда closeOnChange отключается когда enableTime включен. Мы намеренно не даём это делать в DateInput?

Если подумать, то кнопка "Готово" затем и нужна, чтобы иметь возможность закрыть календарь. Если её намеренно прячут, то остаётся только закрытие по клику снаружи, как если бы closeOnChange было false у календаря без времени.

Возможно, если кто-то хочет скрыть эту кнопку, значит он хочет сделать закрытие и применение по-другому. Возможно обернув календарь во что-то и добавив свою кнопку. Другое дело, такое вряд ли получится сделать в DateInput. Возможно в DateInput стоит убрать возможность скрытия этой кнопки

❌ Patch

Не удалось автоматически применить исправление на ветке 7.0-stable.

Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core

Чтобы изменение попало в ветку 7.0-stable, выполните следующие действия:

1. Создайте новую ветку от 7.0-stable и примените изменения используя cherry-pick

git stash # опционально
git fetch origin 7.0-stable
git checkout -b patch/pr7736 origin/7.0-stable

git cherry-pick --no-commit 5775640781504371c739f6c370bdbc903ebf0c86
git checkout HEAD **/__image_snapshots__/*.png
git diff --quiet HEAD || git commit --no-verify --no-edit

2. Исправьте конфликты, следуя инструкциям из терминала
3. Отправьте ветку на GitHub и создайте новый PR с веткой 7.0-stable (установка лейбла не требуется!)

git push --set-upstream origin patch/pr7736
gh pr create --base 7.0-stable --title "patch: pr7736" --body "- patch #7736"

❌ Patch

Не удалось автоматически применить исправление на ветке v6.

Дальнейшие действия выполняют контрибьютеры из группы @VKCOM/vkui-core

Чтобы изменение попало в ветку v6, выполните следующие действия:

1. Создайте новую ветку от v6 и примените изменения используя cherry-pick

git stash # опционально
git fetch origin v6
git checkout -b patch/pr7736 origin/v6

git cherry-pick --no-commit 5775640781504371c739f6c370bdbc903ebf0c86
git checkout HEAD **/__image_snapshots__/*.png
git diff --quiet HEAD || git commit --no-verify --no-edit

2. Исправьте конфликты, следуя инструкциям из терминала
3. Отправьте ветку на GitHub и создайте новый PR с веткой v6 (установка лейбла не требуется!)

git push --set-upstream origin patch/pr7736
gh pr create --base v6 --title "patch: pr7736" --body "- patch #7736"